-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Integrate jest and spectron #28
Conversation
@SerhatKayaa thank you for doing this fix for #27. Could you also include an update to the package.json to add the new dev dependencies of Jest and Spectron as well? I can resolve any issues that come up around the package-lock.json in my own commit (those get ugly in PRs). Also please include a newline at the end of the new test file. |
…tronForce into setup-test-framework#27
Thank you for continuing to work on this. Looks like this draft now moves several of the app files into a |
Sorry for late response, Main src folder is general approach in javascript world. I think you should seperate your folder in src directory.I tried that i change the jest rootDirectory then i get an error without src folder, I'll share you two different electron app, you can look at that project structure. Thank you |
Sure, although there are plenty of electron projects that elect to use |
I think,it's okey right now, can you check this code ? |
The current PR seems to delete all the main project files. |
You can read more via this link
https://circleci.com/blog/electron-testing/
if you test your code, just use npm run test
i left you an example in the test